Skip to content

fix(web): 프로필 이미지 컴포넌트 일부 스타일 디자인 시안과 동일하게 수정#508

Open
enunsnv wants to merge 5 commits intomainfrom
fix/style
Open

fix(web): 프로필 이미지 컴포넌트 일부 스타일 디자인 시안과 동일하게 수정#508
enunsnv wants to merge 5 commits intomainfrom
fix/style

Conversation

@enunsnv
Copy link
Copy Markdown
Contributor

@enunsnv enunsnv commented May 1, 2026 •
edited
Loading

관련 이슈

  • 프로필 이미지 svg 에셋 Figma 디자인과 상이하여 변경 필요
  • 프로필 테두리 색상 또한 Figma 디자인과 상이하여 변경 필요

작업 내용

Before After
before after 
showMentorBadge ? "border-2 border-primary-2" : "border-none"
  • 멘토 배지가 있는 경우 primary-2 색상의 테두리가 적용되도록 수정
  • 멘토 배지가 없는 경우 기본 회색 테두리를 제거하도록 수정

기존

@enunsnv enunsnv requested review from manNomi and wibaek as code owners May 1, 2026 05:10
@vercel
Copy link
Copy Markdown

vercel Bot commented May 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
solid-connect-web-admin Ready Ready Preview, Comment May 1, 2026 5:10am
solid-connection-web Ready Ready Preview, Comment May 1, 2026 5:10am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 1, 2026 •
edited
Loading

Walkthrough

이번 변경 사항은 환경 설정과 UI 스타일링에 관한 작은 업데이트들로 구성되어 있습니다.

  1. 문서 정리

    • .env.guide.md에서 AI Inspector 기능의 환경 변수 등록 가이드 섹션을 제거했습니다. Web 런타임 설정과 GitHub Actions Worker 설정에 관한 39줄의 문서가 삭제되었습니다.

  2. 환경 변수 추가

    • .env.preview에 NEXT_PUBLIC_COOKIE_LOGIN_ENABLED=true 환경 변수를 새로 추가했습니다. 미리보기 환경에서 쿠키 기반 로그인을 활성화하기 위한 변수입니다.

  3. 스타일 업데이트

    • ProfileWithBadge.tsx 컴포넌트의 프로필 이미지 래퍼 스타일을 조정했습니다. 멘토 배지 표시 여부에 따라 border 클래스를 변경했으며, 논리나 렌더링 동작에는 변화가 없습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • wibaek
  • manNomi
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed PR 설명이 관련 이슈, 작업 내용, 변경 사항을 충분히 포함하고 있으며 비포/애프터 이미지로 시각적 증거를 제시하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed PR 제목이 변경 내용의 주요 목적과 일치합니다. 프로필 이미지 컴포넌트의 스타일을 디자인 시안과 맞추는 것이 PR의 핵심입니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/style

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the web label May 1, 2026
@enunsnv enunsnv changed the title Fix/style fix(web): 프로필 이미지 컴포넌트 일부 스타일 디자인 시안과 동일하게 수정 May 1, 2026
@Designer-Su
Copy link
Copy Markdown

Designer-Su commented May 1, 2026

안녕하세요, 은정님 고생해주셔서 감사합니다 :)

저의 코멘트는 아래와 같습니다.

  1. 프로필 설정이 안되어있을때 기존 사람 이미지에서 K100 색상이 현재 디자인인데, 해당 디자인으로 들어가는것일까요?
  2. 카드 채널 컴포넌트를 조금 변경하였습니다. (글자간의 패딩값, 글자 색상 K900으로 변경, 채널 카드에서 합격레시피 넣기 등등 자세한 컴포넌트 변경은 아래 피그마에서 확인 부탁드립니다.

피그마 링크:https://www.figma.com/design/TGJs0mxQwRoGVsVjCOIHi7/SLCN-Design-System--App-?node-id=1597-10703&t=2HY0M7gOHRTLx3hH-1
3. 작업해주신 페이지의 패딩값이 디자인과 다른 부분이 있는 것 같습니다. 참고용으로 디자인 캡쳐와 링크 전달드립니다 :)

이미지: Screenshot 2026-05-01 at 6 07 13 PM

피그마 링크: https://www.figma.com/design/TGJs0mxQwRoGVsVjCOIHi7/SLCN-Design-System--App-?node-id=3232-41710&t=2HY0M7gOHRTLx3hH-1

  1. 멘티가 보는 페이지에서의 패딩값도 멘토 페이지의 수정 사항까지 같이 반영 부탁드립니다!

보기시 어려우신 부분있으면 언제나 미팅 요청 주세요!
감사합니다 :)

manNomi
manNomi approved these changes May 1, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@manNomi manNomi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

border 색상이 잘못들어가있었군요
수경님 말씀주신것만 추가로 반영해주셔도 감사할것 같습니다!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@manNomi manNomi manNomi approved these changes

@wibaek wibaek Awaiting requested review from wibaek wibaek is a code owner

Assignees

No one assigned

Labels

web

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@enunsnv @Designer-Su @manNomi